-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add prepared geometry wrapper #105
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Anshul Singhvi <[email protected]>
What are your thoughts about the preparation interface? We need to be able to take a reliable path for preparations which might have |
I'm not sure really! I'm hoping that will be easier when there is a concrete PR using it. Probably we will need to make some changes to some of this too. |
Sampling could use this - e.g. A current list of preparations for #184 would be:
I guess each preparation would have a specialized function? Or we can override e.g. area on multipolygons which have the SubpolygonArea preparation. Not sure how to dispatch on only prepared geoms which have some preparation, nor how to define an order of priority for that dispatch. |
Needs tests
Fixes #103